messageTray: Remove _lastNotificationRemoved()

No Source subclass used it to do anything special, and with sources
no longer having any UI representation on their own, doing anything
else isn't useful either, so just kill off that hook.

https://bugzilla.gnome.org/show_bug.cgi?id=744850

diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index 4baed6a79af463495e3eb4bbefb70cf26660ab04..4bf25805ef2ac7870b295f4a9c447b21eb903128 100644 (file)
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -1304,7 +1304,7 @@ const Source = new Lang.Class({
 
         this.notifications.splice(index, 1);
         if (this.notifications.length == 0)
-            this._lastNotificationRemoved();
+            this.destroy();
 
         this.countUpdated();
     },
@@ -1357,11 +1357,6 @@ const Source = new Lang.Class({
                 this.notifications[i].destroy();
 
         this.countUpdated();
-    },
-
-    // Default implementation is to destroy this source, but subclasses can override
-    _lastNotificationRemoved: function() {
-        this.destroy();
     }
 });
 Signals.addSignalMethods(Source.prototype);

diff --git a/js/ui/notificationDaemon.js b/js/ui/notificationDaemon.js
index aedda638fa9e77986f6a5ef13dda56e012ac0f43..8d470c9cfbc2226dfb4bf89fa9b79810e25b4d2a 100644 (file)
--- a/js/ui/notificationDaemon.js
+++ b/js/ui/notificationDaemon.js
@@ -551,10 +551,6 @@ const FdoNotificationDaemonSource = new Lang.Class({
         this.destroyNonResidentNotifications();
     },
 
-    _lastNotificationRemoved: function() {
-        this.destroy();
-    },
-
     openApp: function() {
         if (this.app == null)
             return;